Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Puppet 6.6 evaltrace for progress #249

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jan 21, 2020

9fc5fd3 disabled the progress bar monkey patch on Puppet 6. Puppet 6.6.0 started to mention the progress when evaluating resources which can be used to update the progress bar.

This is currently a draft since the actual line message still needs to be extracted and causes the tests to fail.

@ehelms
Copy link
Member

ehelms commented Jan 21, 2020

Whats the check that ensures this still works with Puppet 5?

@ekohl
Copy link
Member Author

ekohl commented Jan 22, 2020

I should have provided https://tickets.puppetlabs.com/browse/PUP-6465 as context. That links to puppetlabs/puppet@dd01573. As you can see, Starting to evaluate the resource changed to Starting to evaluate the resource (%{progress} of %{total}). I probably won't have time to finish this soon, but wanted a quick PoC to see the feasibility of it for a presentation.

9fc5fd3 disabled the progress bar
monkey patch on Puppet 6. Puppet 6.6.0 started to mention the progress
when evaluating resources which can be used to update the progress bar.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants